role skill の発火条件を明確化 - #106
Merged
Merged
Conversation
理由: role skill の発火条件を明確にし、適切な skill 選択を安定させる。 変更: 各 role skill の description と design-skill の記述を更新し、対応する evals を追加・更新した。 検証: quick_validate、JSON、forward-test 24/24、deploy を実行済み。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
role skill の発火条件を明確化
Summary
role-* の入口 skill が、調査・計画・助言・実装・リファクタリング・レビュー・Git 操作・機械作業のどの場面で使われるかを metadata だけで判別できるようにしました。
主な変更点:
やったこと
role-advisor、role-gardener、role-implementer、role-planner、role-refactor、role-reviewer、role-scouter、role-workerの description を更新evals/evals.jsonを各 skill に追加docs/design-skill.mdに、routing 条件は入口 description、実行手順と判断基準は playbook に置く方針を明記動作確認
quick_validate.py: 8/8 成功git diff --check: 成功make deploy-dry-run: 成功make deploy: 成功レビュー & 動作確認 チェックリスト
推奨テスト計画:
SKILL.mdの description と対応する eval prompt を比較するmake deploy-dry-runで配布対象を確認するその他気になることや相談ごと
なし